azurerm_storage_account - Support for sas_policy
#19222
Conversation
| Type: pluginsdk.TypeString, | ||
| Required: true, | ||
| ValidateFunc: validation.StringInSlice([]string{ | ||
| // There is no definition of this enum in the Track1 SDK due to: https://github.com/Azure/azure-sdk-for-go/issues/14589 |
There was a problem hiding this comment.
that issue was closed as wont't fix - has it been added to the swagger or should we be opening an issue on the rest api specs?
There was a problem hiding this comment.
That is not a swagger issue, but a Track1 SDK issue. As they are not support track1 SDK, they closed that issue. This comment is to explain why there isn't an enum defined in the SDK, not meant to be a TODO. If there are more than one enum defined for that attribute in later API, then there will be enums defined in the SDK.
There was a problem hiding this comment.
so the swagger is correct?
also is there a reason for this to be required with a single value? we could make it optional with a default?
There was a problem hiding this comment.
Yes, swagger is correct. Good idea, will make the change then.
There was a problem hiding this comment.
perfect thanks for checking - this'll be fine and be fixed by the pandora SDK then
| Type: pluginsdk.TypeString, | ||
| Required: true, | ||
| ValidateFunc: validation.StringInSlice([]string{ | ||
| // There is no definition of this enum in the Track1 SDK due to: https://github.com/Azure/azure-sdk-for-go/issues/14589 |
There was a problem hiding this comment.
perfect thanks for checking - this'll be fine and be fixed by the pandora SDK then
|
This functionality has been released in v3.32.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you! |
|
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Fix #19188
Test